Skip to content

Added code to enable Dynamic Key Guest Secure Boot#925

Open
Krishan-Saraswat wants to merge 1 commit into
open-power:masterfrom
Krishan-Saraswat:dkgsb
Open

Added code to enable Dynamic Key Guest Secure Boot#925
Krishan-Saraswat wants to merge 1 commit into
open-power:masterfrom
Krishan-Saraswat:dkgsb

Conversation

@Krishan-Saraswat

Copy link
Copy Markdown

Dynamic Key Guest Secure Boot needs kernel config check, setting platform keys, checking signatures of GRUB and Kernel, added code to enable and disable DKGSB.

@PraveenPenguin PraveenPenguin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Krishan-Saraswat i have just glance .. i see we have scope of reusability of code .. let me know your thought

Comment thread testcases/OpTestDKGSB.py Outdated
Comment thread testcases/OpTestDKGSB.py Outdated
Comment thread testcases/OpTestDKGSB.py Outdated
Comment thread testcases/OpTestDKGSB.py Outdated
Comment thread testcases/OpTestDKGSB.py Outdated
Comment thread testcases/OpTestDKGSB.py Outdated
Comment thread testcases/OpTestDKGSB.py Outdated
Dynamic Key Guest Secure Boot needs kernel config check,
setting platform keys, checking signatures of GRUB and Kernel,
added code to enable and disable DKGSB.

Signed-off-by: Krishan Gopal Saraswat <krishang@linux.ibm.com>
@abdhaleegit

Copy link
Copy Markdown
Collaborator

@nasastry please review

@PraveenPenguin PraveenPenguin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PraveenPenguin

Copy link
Copy Markdown
Collaborator

@Krishan-Saraswat can you please post latest run logs as we have some chanegs happend in API front what to double sure all are good .. if not we need to fix that using laternate API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants